Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rails 5 fix for undefined method select_recurring #115

Closed
wants to merge 2 commits into from

Conversation

sahild
Copy link

@sahild sahild commented Oct 31, 2016

After Rails 5 upgrade, gem stopped working with this error:
ActionView::Template::Error (undefined method select_recurring' for #<#<Class:0x007fa3ae96a600>:0x007fa3ae9681c0>). at this line = f.select_recurring :current_custom_rule, []`

I added this quick fix to make this method available for rails 5.
If you can release a version with this fix i will be able to use this gem again from 'rubygems.org'

@sahild
Copy link
Author

sahild commented Oct 31, 2016

@forrest @nathany @afhammad If you can please have a look at this fix to make this gem work for rails 5.
Thanks and Regards
Sahil Dhankhar

@nathany
Copy link
Contributor

nathany commented Nov 1, 2016

@sahild Thanks for submitting a patch. FYI, I'm no longer involved on this project.

@ghost
Copy link

ghost commented Apr 6, 2017

Any update on this?

@stephen-puiszis
Copy link
Contributor

stephen-puiszis commented Jul 19, 2017

👍 What's going on with this?

@msxavi
Copy link

msxavi commented Aug 2, 2017

So is anyone able to merge this?

@Frank004
Copy link

Frank004 commented Sep 14, 2017

any merge ??

and @sahild thnk for the pr using this until it merge.

@angantyr
Copy link

+1

gustavrannestig added a commit to prototypsthlm/recurring_select that referenced this pull request Jan 12, 2018
This should allow usage of this gem with rails 5. Note that there is an unmerged to the original project at: gregschmit#115
@terencedignon
Copy link

+1

@jclif
Copy link

jclif commented Mar 14, 2018

Is anyone still working on this project?

@CodingAnarchy
Copy link

Is this repository no longer maintained?

@luisenrike
Copy link

Merge this please :(

@chris-hut
Copy link
Contributor

Closed by #122

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.